Skip to content

Make logging consistent for matrixRTC #4788

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Apr 10, 2025

Conversation

toger5
Copy link
Contributor

@toger5 toger5 commented Apr 4, 2025

This will make the logs look like this:
image

Checklist

  • Tests written for new code (and old code if feasible).
  • New or updated public/exported symbols have accurate TSDoc documentation.
  • Linter and other CI checks pass.
  • Sign-off given on the changes (see CONTRIBUTING.md).

@toger5 toger5 added T-Task Tasks for the team like planning T-Enhancement and removed T-Task Tasks for the team like planning labels Apr 7, 2025
@toger5 toger5 force-pushed the valere/matrix_rtc_to_device_keys branch from 584f4f6 to 4ce419d Compare April 7, 2025 08:59
@toger5 toger5 force-pushed the toger5/consistent-matrix-rtc-logging branch 3 times, most recently from 5f5d48a to 25afe74 Compare April 7, 2025 11:14
@toger5 toger5 force-pushed the toger5/consistent-matrix-rtc-logging branch 2 times, most recently from 4929567 to ea55bff Compare April 7, 2025 11:26
Copy link
Member

@BillCarsonFr BillCarsonFr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice the easy filtering is very usefull

@BillCarsonFr BillCarsonFr force-pushed the valere/matrix_rtc_to_device_keys branch from 499c56e to 0a587e9 Compare April 7, 2025 12:55
@toger5 toger5 force-pushed the toger5/consistent-matrix-rtc-logging branch from ea55bff to cd61465 Compare April 8, 2025 15:41
@toger5 toger5 force-pushed the toger5/consistent-matrix-rtc-logging branch from cd61465 to f662b52 Compare April 8, 2025 16:17
@toger5 toger5 force-pushed the toger5/consistent-matrix-rtc-logging branch from f662b52 to 40f3cee Compare April 9, 2025 10:22
@toger5 toger5 force-pushed the valere/matrix_rtc_to_device_keys branch from 3178269 to 778db8c Compare April 9, 2025 11:09
Base automatically changed from valere/matrix_rtc_to_device_keys to develop April 10, 2025 08:41
@toger5 toger5 force-pushed the toger5/consistent-matrix-rtc-logging branch from 40f3cee to 3cec377 Compare April 10, 2025 10:27
@toger5 toger5 marked this pull request as ready for review April 10, 2025 10:27
@toger5 toger5 requested a review from a team as a code owner April 10, 2025 10:27
@toger5 toger5 requested a review from AndrewFerr April 10, 2025 10:27
@BillCarsonFr BillCarsonFr added this pull request to the merge queue Apr 10, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 10, 2025
@toger5 toger5 added this pull request to the merge queue Apr 10, 2025
Merged via the queue into develop with commit e3a3a52 Apr 10, 2025
30 checks passed
@toger5 toger5 deleted the toger5/consistent-matrix-rtc-logging branch April 10, 2025 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants